feat: search current working directory for config file #1464
Conversation
| # Look into the home directory | ||
| if pyiceberg_home_config := _load_yaml(os.path.expanduser("~")): | ||
| return pyiceberg_home_config | ||
| # Directories to search for the configuration file |
There was a problem hiding this comment.
nit: i like the comment on search order "PYICEBERG_HOME, home directory, then current directory"
There was a problem hiding this comment.
I added a new line to show this in 2bd6913, is this what you are looking for? I just added back this comment underneath.
mkdocs/docs/configuration.md
Outdated
There was a problem hiding this comment.
there are a couple other places where pyiceberg.yaml is referenced in the docs
https://grep.app/search?q=pyiceberg.yaml&filter[repo][0]=apache/iceberg-python&filter[path][0]=mkdocs/docs/
There was a problem hiding this comment.
Nice finds! That grep tool is pretty neat. I just made some corrections to these in 3ebbb8c.
There was a problem hiding this comment.
There was a problem hiding this comment.
I tried adding a test here, but I wonder if there are opportunities to clean it up.
tests/utils/test_config.py
Outdated
|
|
||
|
|
||
| @pytest.mark.parametrize( | ||
| "config_location, config_content, expected_result", |
There was a problem hiding this comment.
nit: config_content and expected_result are always the same, we can use one of them, write to the file and verify that it reads back the same
|
There's another test that looks to do something similar to this newer test. Are both needed? iceberg-python/tests/utils/test_config.py Lines 58 to 65 in 0e5086c |
Resolves #1333
Adds the current working directory to the search path for the
.pyiceberg.yamlfile.As it is now, the file is searched in the following order:
PYICEBERG_HOMEenvironment variableI'm unsure if people would like to have 2 and 3 swapped. In either case, users can still override this with the environment variable.